Skip to content

Conversation

pebeto
Copy link
Member

@pebeto pebeto commented Aug 17, 2025

After the release of mlflow v3, some data structures changed. Also, there's a new way to log model information using the logmodel function, passing MLmodel object in JSON format.

@pebeto pebeto added the enhancement New feature or request label Aug 17, 2025
@pebeto
Copy link
Member Author

pebeto commented Aug 17, 2025

@ablaom, after the implementation of logmodel, looks like there's no plan for the mflow team to expand the artifact storage functionality (making it official).
The previous mentioned function needs the models being transformed into the MLmodel format. There's more information here. If there's any plan to make use of it, MLJ.jl needs to implement a way to transform the machines (or models) into this format.

@pebeto pebeto force-pushed the pairing-mlflow-v3 branch 2 times, most recently from fb8f42b to f3c73e4 Compare August 17, 2025 18:53
@pebeto pebeto merged commit eb70d6d into main Aug 18, 2025
3 checks passed
@ablaom
Copy link
Member

ablaom commented Aug 18, 2025

Are you saying the previous artifact functionality has changed in a way that breaks the current MLJFlow.jl ? Or just that this API may not be supported in a future release of mlflow?

I would support work to adopt the new logmodel way of doing things. I can review parts of any work, as it applies to the MLJ API, but have pretty limited bandwidth.

@ablaom
Copy link
Member

ablaom commented Aug 18, 2025

Thanks for you work on this update, by the way!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants